🎨 Palette: Add toast notification for missing CLI#155
Conversation
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
1 participant
🎨 Palette: Add toast notification for missing CLI
💡 What
Added a one-time toast notification (
vscode.window.showErrorMessage) when backgroundexecFilecalls fail because theledgermind-mcpCLI is not found (ENOENT).🎯 Why
Previously, if the
ledgermind-mcpCLI wasn't installed, the extension would silently log an error to the output channel and turn the status bar red. Users would not know they needed to runpip install ledgermind. This change surfaces the missing dependency with an actionable error message.📸 Before/After
Before: Status bar turns red, error hidden in output logs.
After: Toast notification appears: "LedgerMind CLI not found. Please install it using 'pip install ledgermind'"
♿ Accessibility
The toast notification provides immediate, screen-reader accessible feedback about why the background operation failed, removing the need to hunt through logs to discover a missing dependency.
PR created automatically by Jules for task 5700501197592085573 started by @sl4m3